Skip to content

Conversation

jLynx
Copy link
Owner

@jLynx jLynx commented Nov 15, 2024

Description

Improves the macOS app update process by adding proper handling for permission-denied scenarios and implementing a more robust extraction mechanism.

Changes

  • Creates separate temporary directories for backup and extraction to better manage the update process
  • Adds proper directory creation for extracted files to prevent path-related issues
  • Implements fallback to AppleScript with admin privileges when standard file operations fail due to permissions
  • Improves error handling and cleanup in case of failures during the update process
  • Maintains atomic update operations to prevent partial updates

Why

The current implementation could fail on macOS when the app lacks sufficient permissions to modify its own directory, which is common in certain installation locations. This update provides a more reliable solution by:

  1. Properly handling permission-denied scenarios
  2. Using admin privileges when necessary
  3. Ensuring clean rollback in case of failures

Testing

  • Tested app updates in various installation locations on macOS
  • Verified successful updates both with and without admin privileges
  • Confirmed proper cleanup in error scenarios
  • Tested rollback functionality

Please test thoroughly on macOS systems with different installation locations and permission scenarios.

Copy link

github-actions bot commented Nov 15, 2024

Package Changes Through d5699a5

No changes.

Add a change file through the GitHub UI by following this link.


Read about change files or the docs at github.com/jbolda/covector

jLynx and others added 28 commits November 15, 2024 13:22
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
…ps#2072)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
…2084)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
…ri-apps#2096)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
renovate bot and others added 30 commits January 18, 2025 20:38
…2331)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
…ri-apps#2337)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
These will have a bunch of default features enabled, so let's
preemptively disable them.
…2346)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
…#2352)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
…apps#2359)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
…ri-apps#2363)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: FabianLars <[email protected]>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.